-
Notifications
You must be signed in to change notification settings - Fork 150
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Support running as a Docker container #33
Support running as a Docker container #33
Conversation
88152ef
to
36588b7
Compare
Fixed merge conflicts. |
b4865d0
to
fdb079c
Compare
Hi @PlasmaPower , Would you be willing to submit the custom port functionality as a separate PR? |
Done with #39. |
8161818
to
989aa66
Compare
Also, this removes the need for Docker accepting TCP connections (so you don't have to modify the docker service any more). |
989aa66
to
a60cb87
Compare
I'm planning to add support for Kubernetes, and I don't think I'll be using the name based DNS system for that. I think it's better to describe this option as internal networking, which describes the what instead of the how.
Hi @PlasmaPower , Would it be possible to re-submit this PR on the develop branch? Thanks! |
Example use:
Note: in a Swarm, shinyproxy must be run on a manager node. This can be accomplished with the
deploy.placement.constraints
section ofdocker-compose.yml
.application.yml
:Edit:
internal-networking
option was originallyname-networking
, see b4865d0.I also added support for a custom port for apps, since I was working on much of the same code. I can move that into a separate PR if you'd prefer.Edit: separated into #39.